-
Notifications
You must be signed in to change notification settings - Fork 192
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Lightning CSS to replace postcss build commands #191
base: main
Are you sure you want to change the base?
Conversation
Visit the preview URL for this PR (updated for commit 438df24): https://open-props--pr191-use-parcel-css-540c65nl.web.app (expires Wed, 16 Mar 2022 05:46:28 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 |
much faster builds on this 2010 machine 👍🏻 |
Hey @argyleink! Anything you need help with to get this in? 😄 |
i dont think there's any technical stuff in the way, it's me mostly ironing it all out and doing all the cross checks to ensure to regressions 🙂 thanks for checkin in! |
just spent an hour doing builds and testing/inspecting output! there's a lot of great things going on in this, speed being a huge one. having the logical properties syntax lowered, even place-items, are adding lots of lines of code. makes me want to bundle 2 versions and do the like, modern css vs syntax lowered css builds. if i offer them on the cdn or via exports, they could pick which they snag. i don't see any ways of specifying via cli options to not lower logical props? feature request 🙂 btw though, the syntax lowering of logical props is super impressive and rad. i love the syntax lowering of color! love the clean output. it looks like i could switch to parcel for just builds of the basic files right away, but not normalize since it almost doubled from logical property lowering. but, it's so fast. the main consumed files are mostly values, not logical properties, and it's really only a few lines of change to package.json. i dont think i'd want to juggle both postcss and a parcel tool, but maybe i do! will think and check more. |
Logical properties lowering is based on the targets specified to the CLI. You have "last 2 versions", which would include eg ie 10 and 11. Usually it's better to do a browserslist query based on market share percentage. You can play around here: https://browserslist.dev |
todo: try with the latest Lightning release |
Visit the preview URL for this PR (updated for commit 0379e9d): https://open-props--pr191-use-parcel-css-92b626xs.web.app (expires Wed, 27 Sep 2023 05:37:22 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 32524ac481f54edda55dc959fa4614cf1acc8c11 |
@argyleink is this still something you'd want for OP? If so, my PR you recently merged would need an update to use lightningcss. I'm happy to make those changes once/if this is merged. |
i've been testing it more in OPv2 branch, and still not entirely sure I'll go with it in the end. but thanks for checking and the offer! for now I'd say hold off 🙂 |
todo
this proves the groundwork though, and given the output is smaller, than this PR is a go 👍🏻